Skip to content

Incorporate pull request rails/jquery-ujs#377#160

Closed
marnen wants to merge 1 commit intorails:masterfrom
marnen:more-form-inputs
Closed

Incorporate pull request rails/jquery-ujs#377#160
marnen wants to merge 1 commit intorails:masterfrom
marnen:more-form-inputs

Conversation

@marnen
Copy link

@marnen marnen commented Jun 9, 2014

See rails/jquery-ujs#377 for full discussion; this fixes handling of form inputs when the form attribute instead of a <form> element is used.

@rafaelfranca
Copy link
Member

Thank you for the PR. We will incorporate as soon it is merged there.

@marnen
Copy link
Author

marnen commented Jun 9, 2014

That works. I wasn't sure how to handle a pull request across 2 repositories.

@marnen
Copy link
Author

marnen commented Jul 8, 2014

@rafaelfranca Now that rails/jquery-ujs#377 is merged, should I reopen this?

@lucasmazza
Copy link
Contributor

@marnen we do not accept PRs updating the vendored JavaScript files. The jquery-ujs change will be incorporated here once we cut a new release of the jquery-ujs project.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants